Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add pipefail to space_robots build so that it fails on error #76

Merged
merged 3 commits into from
Aug 17, 2023

Conversation

mkhansenbot
Copy link
Contributor

Adding set -eo pipefail to the space_robots workflow to fix #75

@mkhansenbot mkhansenbot marked this pull request as draft August 14, 2023 16:31
@mkhansenbot mkhansenbot marked this pull request as ready for review August 15, 2023 00:23
@mkhansenbot
Copy link
Contributor Author

This fails the space_robots build as expected!

@mkhansenbot
Copy link
Contributor Author

Also updated moveit2/build.sh and spaceros/build.sh to ensure the scripts fail if the build fails

@mkhansenbot
Copy link
Contributor Author

Rebasing to main now that #58 is merged and fixed the build, this should now pass also

@mkhansenbot
Copy link
Contributor Author

@EzraBrooks - can you review and approve?

Copy link
Member

@EzraBrooks EzraBrooks left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. always one of the first low-hanging fruits I attack on shell scripts too :)

@EzraBrooks EzraBrooks merged commit c60ab97 into space-ros:main Aug 17, 2023
4 checks passed
eholum pushed a commit to eholum/docker that referenced this pull request Jun 9, 2024
Following a minimal set of rules to track changes, associate them to milestones
and releases, and determine the process of accepting them, will help us comply
with processes existing at NASA (e.g., NPR7150.2D) and other organizations for
Space ROS to be accepted in space software applications.

This commit documents in the CONTRIBUTING file the process agreed upon by the
Space ROS Technical Committee.
eholum pushed a commit to eholum/docker that referenced this pull request Jun 9, 2024
To increase visibility of the contributing guidelines and facilitate early
adoption by potential contributors, this commit adds a link to the
CONTRIBUTING.md file from the top-level README.
eholum pushed a commit to eholum/docker that referenced this pull request Jun 9, 2024
eholum pushed a commit to eholum/space-ros that referenced this pull request Jun 26, 2024
Following a minimal set of rules to track changes, associate them to milestones
and releases, and determine the process of accepting them, will help us comply
with processes existing at NASA (e.g., NPR7150.2D) and other organizations for
Space ROS to be accepted in space software applications.

This commit documents in the CONTRIBUTING file the process agreed upon by the
Space ROS Technical Committee.
eholum pushed a commit to eholum/space-ros that referenced this pull request Jun 26, 2024
To increase visibility of the contributing guidelines and facilitate early
adoption by potential contributors, this commit adds a link to the
CONTRIBUTING.md file from the top-level README.
eholum pushed a commit to eholum/space-ros that referenced this pull request Jun 26, 2024
eholum pushed a commit to eholum/space-ros that referenced this pull request Jun 26, 2024
Add pipefail to space_robots build so that it fails on error
eholum pushed a commit to eholum/space-ros that referenced this pull request Jun 26, 2024
Add pipefail to space_robots build so that it fails on error
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

GHA CI is being falsely marked as passing even though the space_robots build fails
2 participants